Skip to content

Comments

Blewis/server models#774

Merged
blewisCycle merged 4 commits intomainfrom
blewis/server-models
Dec 23, 2025
Merged

Blewis/server models#774
blewisCycle merged 4 commits intomainfrom
blewis/server-models

Conversation

@blewisCycle
Copy link
Collaborator

@blewisCycle blewisCycle commented Dec 23, 2025

  • Update naming of ProviderServer to ServerModels to more accurately represent the information of that endpoint
  • Update server POST endpoint to new platform structure
  • Rename server POST to updateServers to represent that it is designed to create multiple servers at once

@mattoni mattoni added the breaking A breaking change to the API label Dec 23, 2025
post:
operationId: "createServer"
operationId: "createServers"
summary: Create Server
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update title to match operation ID

"$ref": ../clusters/Cluster.yml
models:
"$ref": "../../includes/ServerModelIncludes.yml"
"$ref": "../../includes/ProviderServerModelsIncludes.yml"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was singular for a single provider server model before. Should this be the singular version?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No. Includes are plural, I standardized. There were two identical includes in the repo. I removed the incorrecly named one.

@blewisCycle blewisCycle merged commit 949d61b into main Dec 23, 2025
1 check passed
@blewisCycle blewisCycle deleted the blewis/server-models branch December 23, 2025 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking A breaking change to the API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants